Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bot IDs as inputs for generating access tokens #3

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

richtja
Copy link
Contributor

@richtja richtja commented Feb 16, 2024

This is a fix for avocado-project tools which use MR. Avocado bot for manipulation with issue and PR. This tool needs MR. Avocados IDs, which is stored in Avocado organization as secrets. Unfortunately, the composite actions cannot use organization secrets, therefore we need to pass them as inputs to the tool. This commit updates the avocado-project action to use MR. Avocados IDs as inputs.

This is a fix for avocado-project tools which use MR. Avocado bot for
manipulation with issue and PR. This tool needs MR. Avocados IDs, which is
stored in Avocado organization as secrets. Unfortunately, the composite actions
cannot use organization secrets, therefore we need to pass them as inputs to
the tool. This commit updates the avocado-project action to use MR. Avocados
IDs as inputs.

Signed-off-by: Jan Richter <[email protected]>
@richtja richtja added the bug Something isn't working label Feb 16, 2024
@richtja richtja self-assigned this Feb 16, 2024
@richtja
Copy link
Contributor Author

richtja commented Feb 16, 2024

The CI failure is, because it takes the action from main and not from the PR with the fix.

Copy link
Contributor

@clebergnu clebergnu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@clebergnu clebergnu merged commit daa9f5d into avocado-framework:main Feb 16, 2024
1 check failed
@richtja richtja deleted the project_workflow_IDs branch February 20, 2024 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants